Skip to content

Conversation

@katinthehatsite
Copy link
Contributor

Related issues

Related to STU-889

Proposed Changes

This PR ensures that we pass Studio site name to be populated in the domain field when they try to create a site on WordPress.com.

Testing Instructions

  • Pull the changes from this branch
  • Navigate to the Sync tab in Studio and open the Sync modal
  • At the bottom of the modal, click on Create a new WordPress.com site
  • Observe that when you are brought to WP.com, the domain name field on WP.com is populated with your Studio site name:
Screenshot 2025-12-02 at 12 10 27 PM

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@katinthehatsite katinthehatsite self-assigned this Dec 2, 2025
@katinthehatsite katinthehatsite requested a review from a team December 2, 2025 11:11
`https://wordpress.com/setup/new-hosted-site?ref=studio&section=studio-sync&showDomainStep&studioSiteId=${ selectedSite.id }`
`https://wordpress.com/setup/new-hosted-site?ref=studio&section=studio-sync&showDomainStep&studioSiteId=${
selectedSite.id
}&new=${ encodeURIComponent( selectedSite.name ) }`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am just wondering how many people actually rename their Studio sites 😅 for this to be useful

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

📊 Performance Test Results

Comparing c6c308a vs trunk

site-editor

Metric trunk c6c308a Diff Change
load 11460.00 ms 10869.00 ms -591.00 ms 🟢 -5.2%

site-startup

Metric trunk c6c308a Diff Change
siteCreation 90913.00 ms 87945.00 ms -2968.00 ms 🟢 -3.3%
siteStartup 36514.00 ms 30406.00 ms -6108.00 ms 🟢 -16.7%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

Copy link
Contributor

@epeicher epeicher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @katinthehatsite! This is a nice touch for users creating sites on WordPress.com, as they will have a hint of possible domain names based on the names of the local site. I have tested it, and it works as expected. LGTM! :shipit:

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants